duckdb: vortex_version() macro - #9787
Conversation
myrrc
added
changelog/feature
Merging this PR will regress 2 benchmarks
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | WallTime | arrow_checked_add_u32_neon[16384] |
13.3 µs | 20.3 µs | -34.37% |
| ❌ | WallTime | mul_i32_nonnull_avx512 |
7.1 µs | 7.9 µs | -10.1% |
| ⚡ | WallTime | words_gather_dispatch_avx2[1024] |
46 ns | 17 ns | ×2.7 |
| ⚡ | Simulation | allocate_drop_arrow[0] |
456.9 ns | 402.7 ns | +13.45% |
| ⚡ | WallTime | words_gather_scalar_avx2[65536] |
9.3 µs | 8.3 µs | +13.07% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing myrrc/duckdb-vortex-versoin (7e37aec) with develop (a13c246)2
Footnotes
-
218 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
-
No successful run was found on
develop(9fdaa38) during the generation of this report, so a13c246 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
joseph-isaacs
left a comment
joseph-isaacs
left a comment
There was a problem hiding this comment.
This is not the vortex version.
It might be a good idea to add a enabled editions func for the writer?
294c384 to
8da8c7b
Compare
|
CI changes in duckdb-vortex to come after we merge this. |
a0565ea to
a9f48ad
Compare
Signed-off-by: Mikhail Kot <mikhail@spiraldb.com>
8943a46 to
d83ecfc
Compare
2 participants
Add vortex_version() duckdb function macro returning Vortex tag this was built from